da656a5492d9e5b3c3a8d281ecc08864fa1f5c83,src/main/java/org/olat/ims/qti21/ui/editor/ItemSessionControlController.java,ItemSessionControlController,formOK,#UserRequest#,222

Before Change


			itemSessionControl.setAllowComment(Boolean.FALSE);
		}
		
		if(allowReviewEl.isOneSelected() && allowReviewEl.isSelected(0)) {
			checkNotNull(itemSessionControl).setAllowReview(Boolean.TRUE);
		} else if(itemSessionControl != null) {
			itemSessionControl.setAllowReview(Boolean.FALSE);
		}
		
		if(showSolutionEl.isOneSelected() && showSolutionEl.isSelected(0)) {

After Change



	@Override
	protected void formOK(UserRequest ureq) {
		ItemSessionControl itemSessionControl = part.getItemSessionControl();//can be null
		
		// need to be first! 
		if(allowSkippingEl.isOneSelected()) {
			if(allowSkippingEl.isSelected(0)) {//yes
				checkNotNull(itemSessionControl).setAllowSkipping(Boolean.TRUE);
			} else if(allowSkippingEl.isSelected(1)) {//no
				checkNotNull(itemSessionControl).setAllowSkipping(Boolean.FALSE);
			} else if(itemSessionControl != null) {//inherit
				itemSessionControl.setAllowSkipping(null);
			}
		}
		
		if(allowCommentEl.isOneSelected()) {
			if(allowCommentEl.isSelected(0)) {
				checkNotNull(itemSessionControl).setAllowComment(Boolean.TRUE);
			} else if(allowCommentEl.isSelected(1)) {
				checkNotNull(itemSessionControl).setAllowComment(Boolean.FALSE);
			} else if(itemSessionControl != null) {
				itemSessionControl.setAllowComment(null);
			}
		}
		
		if(allowReviewEl.isOneSelected()) {
			if(allowReviewEl.isSelected(0)) {
				checkNotNull(itemSessionControl).setAllowReview(Boolean.TRUE);
			} else if(allowReviewEl.isSelected(1)) {
				checkNotNull(itemSessionControl).setAllowReview(Boolean.FALSE);
			} else if(itemSessionControl != null) {
				itemSessionControl.setAllowReview(null);
			}